Skip to content

add I2S and EDMA support for IMX9 platform #97777

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

add I2S and EDMA support for IMX9 platform #97777

wants to merge 14 commits into from
+324 −96

Conversation

@yananyang-nxp
Copy link

@yananyang-nxp yananyang-nxp commented Oct 17, 2025 •
edited by dleach02
Loading

add I2S and EDMA support for IMX9 platform

The depended PR below has been merged:

@github-actions
Copy link

Hello @yananyang-nxp, and thank you very much for your first pull request to the Zephyr project!
Our Continuous Integration pipeline will execute a series of checks on your Pull Request commit messages and code, and you are expected to address any failures by updating the PR. Please take a look at our commit message guidelines to find out how to format your commit messages, and at our contribution workflow to understand how to update your Pull Request. If you haven't already, please make sure to review the project's Contributor Expectations and update (by amending and force-pushing the commits) your pull request if necessary.
If you are stuck or need help please join us on Discord and ask your question there. Additionally, you can escalate the review when applicable. 😊

@github-actions
Copy link

github-actions bot commented Oct 17, 2025 •
edited
Loading

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff

✅ All manifest checks OK

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@github-actions github-actions bot added manifest manifest-hal_nxp DNM (manifest) This PR should not be merged (controlled by action-manifest) labels Oct 17, 2025
@zephyrbot zephyrbot requested a review from dleach02 October 24, 2025 16:34
@dleach02 dleach02 added this to the v4.3.0 milestone Oct 24, 2025
dleach02
dleach02 previously approved these changes Oct 24, 2025
View reviewed changes
mmahadevan108
mmahadevan108 previously approved these changes Oct 24, 2025
View reviewed changes
JiafeiPan
JiafeiPan previously approved these changes Oct 25, 2025
View reviewed changes
Zhiqiang-Hou and others added 14 commits October 25, 2025 10:05
@Zhiqiang-Hou @JiafeiPan
boards: frdm_imx91: fix board dts and pinctrl dtsi format issue
fbb03cb 
Compliance check reports some format issue, so fix it.

Signed-off-by: Hou Zhiqiang <[email protected]>
@yananyang-nxp @JiafeiPan
drivers: dma: dma_mcux_edma: enable MMIO mapping
bcc64c0 
Enable MMIO mapping in dma_mcux_edma driver

Signed-off-by: Yanan Yang <[email protected]>
@yananyang-nxp @JiafeiPan
drivers: mcux_ccm_rev2: add SAI clock support for i.MX9 platform
74ca8f0 
add SAI clock support for i.MX9 platform

Signed-off-by: Yanan Yang <[email protected]>
Signed-off-by: Jiafei Pan <[email protected]>
Fix CI failure: LEADING_SPACE: please, no spaces at the start of a line
Split code format fixing into a new patch
Signed-off-by: Hou Zhiqiang <[email protected]>
@Zhiqiang-Hou @JiafeiPan
drivers: mcux_ccm_rev2: fix code format
d1668d8 
Remove extra space between the '=' and rvalue.

Signed-off-by: Yanan Yang <[email protected]>
Signed-off-by: Hou Zhiqiang <[email protected]>
@Zhiqiang-Hou @JiafeiPan
drivers: i2s: mcux_sai: correct DMA burst length
0286c80 
The burst length should be initialized with 'word_size_bits'.

Signed-off-by: Yanan Yang <[email protected]>
Signed-off-by: Hou Zhiqiang <[email protected]>
@yananyang-nxp @JiafeiPan
drivers: i2s: i2s_mcux_sai: use device model MMIO mapping
d4ac17d 
Change to use device model MMIO mapping APIs.

Signed-off-by: Yanan Yang <[email protected]>
Signed-off-by: Jiafei Pan <[email protected]>
Fix CI failures: Clean unused code
Signed-off-by: Hou Zhiqiang <[email protected]>
@Zhiqiang-Hou @JiafeiPan
drivers: i2s: mcux_sai: fix the type of var num_of_bufs
c79e92a 
Correct the type of num_of_bufs to uint32_t.

Signed-off-by: Hou Zhiqiang <[email protected]>
@Zhiqiang-Hou @JiafeiPan
drivers: i2s: mcux_sai: set the Bit Clock Swap on the feature enabled
325852a 
When the platform has the Bit Clock Swap feature, enable it.

Signed-off-by: Hou Zhiqiang <[email protected]>
@Zhiqiang-Hou @JiafeiPan
drivers: i2s: i2s_mcux_sai: add i.MX9 platform support
809c28d 
Rework the driver to support i.MX9 platforms.

Signed-off-by: Hou Zhiqiang <[email protected]>
Signed-off-by: Yanan Yang <[email protected]>
Signed-off-by: Jiafei Pan <[email protected]>
@yananyang-nxp @JiafeiPan
dts: arm64: imx93: add sai3_2 and edma4_2 node for i2s_mcux_sai driver
5a645e9 
Add sai3_2 and edma4_2 dts node for use with i2s_mcux_sai driver.

Signed-off-by: Yanan Yang <[email protected]>
Fix indent format issue.
Signed-off-by: Hou Zhiqiang <[email protected]>
@yananyang-nxp @JiafeiPan
boards: imx93_evk: add sai3_2 and edma4_2 node for i2s_mcux_sai driver
a21db4c 
add sai3_2 and edma4_2 dts node for use with i2s_mcux_sai driver

Signed-off-by: Yanan Yang <[email protected]>
@yananyang-nxp @JiafeiPan
boards: frdm_imx93: add sai3_2 and edma4_2 node for i2s_mcux_sai driver
18b5dd3 
add sai3_2 and edma4_2 dts node for use with i2s_mcux_sai driver

Signed-off-by: Yanan Yang <[email protected]>
@yananyang-nxp @JiafeiPan
dts: arm64: imx9131: add sai3_2 and edma4_2 node for i2s_mcux_sai driver
eaa92fd 
add sai3_2 and edma4_2 node for use with i2s_mcux_sai driver

Signed-off-by: Yanan Yang <[email protected]>
Fix the indent format issue.
Signed-off-by: Hou Zhiqiang <[email protected]>
@yananyang-nxp @JiafeiPan
boards: frdm_imx91: add sai3_2 and edma4_2 node for i2s_mcux_sai driver
22ea7b7 
add sai3_2 and edma4_2 node for use with i2s_mcux_sai driver

Signed-off-by: Yanan Yang <[email protected]>
@JiafeiPan JiafeiPan dismissed stale reviews from mmahadevan108, dleach02, and themself via 22ea7b7 October 25, 2025 02:06
@JiafeiPan
Copy link
Contributor

This PR has been approved, just rebased to fix confliction

@zephyrbot zephyrbot requested a review from JiafeiPan October 25, 2025 02:08
@sonarqubecloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@JiafeiPan JiafeiPan JiafeiPan approved these changes

@axelnxp axelnxp Awaiting requested review from axelnxp

@bperseghetti bperseghetti Awaiting requested review from bperseghetti

@decsny decsny Awaiting requested review from decsny

@EmilioCBen EmilioCBen Awaiting requested review from EmilioCBen

@iuliana-prodan iuliana-prodan Awaiting requested review from iuliana-prodan

@manuargue manuargue Awaiting requested review from manuargue

@MaochenWang1 MaochenWang1 Awaiting requested review from MaochenWang1

@PetervdPerk-NXP PetervdPerk-NXP Awaiting requested review from PetervdPerk-NXP

@yangbolu1991 yangbolu1991 Awaiting requested review from yangbolu1991

@ZhaoxiangJin ZhaoxiangJin Awaiting requested review from ZhaoxiangJin

@Zhiqiang-Hou Zhiqiang-Hou Awaiting requested review from Zhiqiang-Hou

@anangl anangl Awaiting requested review from anangl

@nordic-krch nordic-krch Awaiting requested review from nordic-krch

@teburd teburd Awaiting requested review from teburd

@sreeramIfx sreeramIfx Awaiting requested review from sreeramIfx

@dleach02 dleach02 Awaiting requested review from dleach02

@mmahadevan108 mmahadevan108 Awaiting requested review from mmahadevan108

@dbaluta dbaluta Awaiting requested review from dbaluta

At least 2 approving reviews are required to merge this pull request.

Assignees

@mmahadevan108 mmahadevan108

@dleach02 dleach02

@decsny decsny

Labels

area: ARM64 ARM (64-bit) Architecture area: Boards/SoCs area: Clock Control area: DMA Direct Memory Access area: I2S platform: NXP MPU platform: NXP NXP

Projects

None yet

Milestone

v4.3.0

Development

Successfully merging this pull request may close these issues.

8 participants

@yananyang-nxp @dleach02 @JiafeiPan @Zhiqiang-Hou @dbaluta @mmahadevan108 @zephyrbot @decsny